Core(utils): type 6 m_ files ahead of the rename, part 2 - #35504

m_error: the helpers are declared before the object that holds them, the
parameters are no longer reassigned, and Error keeps being a function
expression because callers use it with new.

m_date_serialization: the helpers are declared before their callers,
String#match becomes RegExp#exec (no g or y flags), the month of an ISO
string is computed instead of decremented in place, and the results of
the public functions stay any for the callers.

m_selection_filter: the constructor function gets a typed this and its
helpers are declared before the methods that use them.

m_shadow_dom: the hash function keeps its bitwise operations under
reasoned disables; the index loops over DOM collections become for-of.

The tests pass on the code before the typing.

… survived

The overloads type the callbacks of arrays, readonly arrays, nullable arrays and
dictionaries. An empty array literal (never[]) and an untyped {} keep untyped callbacks,
and a loose overload stays last for the callers that pass any.

The callers get type-only fixes: 26 ts-expect-error directives where a declared type does
not match the use, a cast of the scrollbar direction, an annotation of the popup item and
a removed redundant cast. The emitted code of all 17 files is identical.

The overloads return the type of the target merged with the types of the sources, for the
shallow form, the deep form (true first) and the forms where a missing target (false,
null, undefined) makes extend create a new object. An any source gives an any result,
and a loose overload stays last for the callers that spread their arguments.

The callers get type-only fixes: 55 ts-expect-error directives where the merged type is
not the type that the caller declares or uses, two annotations in the pivot grid, an
optional third parameter of foreachNodes and one removed redundant cast; two directives
that the new types made unnecessary are removed. The emitted code of all 33 files is
identical.

deserializeDate returns a date or the unparsed input (never a number), serializeDate
returns the input, a number, a string or null, and getDateSerializationFormat returns a
string, null or undefined.

The callers get type-only fixes: five as string and as Date casts like the ones that the
other callers of these functions already use, and six ts-expect-error directives where the
declared type of the caller is narrower than what the function can return. The emitted
code of all 7 files is identical.

Error is callable and constructible (new errors.Error is used in m_data, filter_builder
and the grid), takes the id and the details and returns a DxError; log takes the id and
the details. The four ts-expect-error directives that waited for Error to be declared as a
constructor are removed, and the unneeded no-unsafe-return disable in m_error goes too.

The callers get type-only fixes: a cast of the id in data/errors, three directives where
a caller spreads untyped arguments or passes an optional code, and seven directives in
the Jest tests that mock Error with an empty object. The emitted code is identical.

The filter builder was typed in DevExpress#35477 after the hub typing was cut: its action config is
the merged object that the action options do not accept. Only a directive is added, the
emitted code is unchanged.

…iles

Eight errors in six files that this branch did not touch, found by the full
eslint run after the rebase onto the newer main (the typed extend and each
make a cast unnecessary, and the typed string makes match a prefer-regexp-exec
error):
- 5 casts of the result of extend to the type that it already has are removed
  (devices, card_view, grid_core widget, toolbar utils, work_space x2)
- animation/fx: String#match with a regexp without the g flag becomes
  RegExp#exec, the same result (the only change of the emitted code here)

…d of ignoring them

The context menu controller calls _getRows and getContextMenuItems of the grid
views. The views declared _getRows as protected, getContextMenuItems was
private in two extenders and not declared in the base, so the two calls had a
ts-expect-error each. Now _getRows is public in the five view classes (the
change parameter of the rows view is optional like in the base class),
ColumnsView declares the optional getContextMenuItems and the two extenders
make it public. The directive of adaptivity_rows_view that expected the
missing declaration is removed too.

The scrollable strategy declared the action handler with a
Record<string, unknown> argument while the component returns a handler that
takes unknown, so the call that merges the action args needed a directive.
The declaration follows the component now.

Three directives of the hub typing and one older directive are removed, 16 are
left. The emitted code is identical.

Copilot review: normalizeLoadResult does not make extra an object. An array
without a second argument gives extra: undefined and any other value of the
second argument is passed through, so the cast to Record<string, unknown> and
the interface that it needed made the contract unsound. utils.ts is back to
the base version, and the one place where the declared LoadResult type is
ahead of the data (an object extra only exists after _processStoreLoadResult
has normalized it) has a ts-expect-error that says so.

The emitted code is identical.

The sortable that is typed in main now passes extend({}, animationConfig, { to })
to animate. The typed extend models a merge as an intersection, so the new to
is intersected with the string | AnimationState of the public animation config
and the result does not fit AnimateConfig. The new to replaces the old one at
run time, so the merged object is an AnimateConfig, and the call says so.

The emitted code is identical.

Copilot review: MergedSource used NonNullable<T>, so extend({}, source) with
source: { value: string } | undefined typed result.value as string, but an
undefined source is skipped at run time and value is absent. A source whose
type contains null, undefined or void now adds Partial<NonNullable<T>>; a
source that is always present keeps its properties required. No caller of
extend needs a change (tsc is clean), and two type tests in m_extend.test.ts
pin both cases (they are checked by tsc, ts-jest does not type-check).

The emitted code is identical.